Skip to content

Conversation

@HumphreyYang
Copy link
Member

Similar to the LLN and CLT lecture, this PR resolves some of the comments proposed this morning. Due to the audio quality, I missed some of the valuable comments from the team. Please kindly comment on this PR so that I can further improve this lecture.

@netlify
Copy link

netlify bot commented Feb 7, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 6d118a6
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/63e32b8125b9960008af6d24
😎 Deploy Preview https://deploy-preview-67--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@HumphreyYang
Copy link
Member Author

@mmcky there seem to be false positives in the line checker:

https://stats.libretexts.org/Bookshelves/Probability_Theory/Probability_Mathematical_Statistics_and_Stochastic_Processes_(Siegrist)/16:_Markov_Processes/16.05:_Periodicity_of_Discrete-Time_Chains#:~:text=A%20state%20in%20a%20discrete,limiting%20behavior%20of%20the%20chain. - Anchor ':~:text=A%20state%20in%20a%20discrete,limiting%20behavior%20of%20the%20chain.' not found

zreferences: line   13) broken    https://doi.org/10.1257/aer.20151684 - 403 Client Error: Forbidden for url: https://pubs.aeaweb.org/doi/10.1257/aer.20151684

Many thanks if you could take a look :)

@jstac
Copy link
Contributor

jstac commented Feb 7, 2023

Thanks @HumphreyYang , it's looking good. Small comments above.

@mmcky
Copy link
Contributor

mmcky commented Feb 8, 2023

thanks @HumphreyYang would you mind to review the merge conflict? It looks like there have been updates made on the main branch.

@github-actions github-actions bot temporarily deployed to commit February 8, 2023 04:51 Inactive
@HumphreyYang
Copy link
Member Author

thanks @HumphreyYang would you mind to review the merge conflict? It looks like there have been updates made on the main branch.

Many thanks! It seems like the main branch has some building issues.

@jstac
Copy link
Contributor

jstac commented Feb 8, 2023

Thanks @HumphreyYang .

@mmcky , please merge when ready (noting the build failure).

@jstac
Copy link
Contributor

jstac commented Feb 16, 2023

@mmcky I couldn't see what the build issue was but I'm going to go ahead and merge this. We can iterate from there.

@jstac jstac merged commit 8a0be8f into main Feb 16, 2023
@jstac jstac deleted the integrate-mc branch February 16, 2023 22:08
HumphreyYang referenced this pull request Feb 19, 2023
Misc edits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants